Skip to content

[ADDED] ProxyRequired user claims property#239

Merged
aricart merged 1 commit intomainfrom
add_require_proxied_user_perm
Aug 8, 2025
Merged

[ADDED] ProxyRequired user claims property#239
aricart merged 1 commit intomainfrom
add_require_proxied_user_perm

Conversation

@kozlovic
Copy link
Member

@kozlovic kozlovic commented Aug 6, 2025

This would be used in the server to enforce that a user connects through a trusted proxy.

Signed-off-by: Ivan Kozlovic ivan@synadia.com

@kozlovic kozlovic marked this pull request as draft August 6, 2025 17:10
@kozlovic kozlovic requested a review from aricart August 6, 2025 17:10
@coveralls
Copy link

coveralls commented Aug 6, 2025

Pull Request Test Coverage Report for Build 16786523851

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 81.072%

Totals Coverage Status
Change from base Build 14495279614: 0.0%
Covered Lines: 2767
Relevant Lines: 3413

💛 - Coveralls

Copy link
Member

@aricart aricart left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

This would be used in the server to enforce that a user connects
through a trusted proxy.

Signed-off-by: Ivan Kozlovic <ivan@synadia.com>
@kozlovic kozlovic force-pushed the add_require_proxied_user_perm branch from 0964917 to 7fdd66f Compare August 6, 2025 19:25
@kozlovic kozlovic changed the title [ADDED] RequireProxied user claims property [ADDED] ProxyRequired user claims property Aug 6, 2025
@kozlovic
Copy link
Member Author

kozlovic commented Aug 6, 2025

@aricart Renamed the field to ProxyRequired

@kozlovic kozlovic marked this pull request as ready for review August 8, 2025 13:31
@kozlovic
Copy link
Member Author

kozlovic commented Aug 8, 2025

@aricart When you get a chance, please merge and maybe release so that I can update the NATS Server PR. Thanks!

@aricart aricart merged commit 953e18b into main Aug 8, 2025
8 checks passed
@aricart aricart deleted the add_require_proxied_user_perm branch August 8, 2025 14:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants